Document SNAP work requirement modeling simplifications and fix mislabeled HR1 parameter#8869
Merged
MaxGhenis merged 1 commit intoJul 5, 2026
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8869 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 5 3 -2
Lines 62 72 +10
=========================================
+ Hits 62 72 +10
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
…beled HR1 parameter - Add documentation to meets_snap_abawd_work_requirements explaining the 3-in-36 steady-state simplification of the 7 CFR 273.24(b) time limit, including regaining provisions and the FY2026 HR1 clock restart. - Add documentation to meets_snap_general_work_requirements noting that voluntary quit / reduced work effort (7 CFR 273.7(j)) is not modeled. - Add comments noting the annual-average-hours proxy for monthly work hour determinations and the annual UC receipt simplification. - Rewrite gov/usda/snap/work_requirements/abawd/in_effect description and label: it gates all HR1 ABAWD changes, not only the homeless and veteran exemption removals. No behavior changes. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
|
Rebased onto current Verification on the rebased branch:
Ready for review. |
bbeadff to
0c26de2
Compare
MaxGhenis
approved these changes
Jul 5, 2026
MaxGhenis
left a comment
Contributor
There was a problem hiding this comment.
Fable review + agent verification: documentation attributes plus the mislabeled in_effect parameter fix, statutory refs verified; 78 tests green on the re-rebase.
This was referenced Jul 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #8866
Documentation/metadata-only PR — no behavior changes.
Changes
meets_snap_abawd_work_requirements.py): adds adocumentationattribute explaining that 7 CFR 273.24(b) disqualifies an ABAWD only after 3 countable months in 36, with regaining provisions (80 hours in 30 days, 273.24(d)) and one bonus 3-consecutive-month allotment (273.24(e)). The model treats noncompliance as immediate monthly ineligibility — a steady-state assumption that overstates ineligibility for new spells, largest in FY2026 as HR1 restarts time-limit clocks.meets_snap_general_work_requirements.py): adds adocumentationattribute noting that 7 CFR 273.7(j) disqualifications are not modeled, since job-separation events are unobservable in cross-sectional survey data.gov/usda/snap/work_requirements/abawd/in_effect.yaml): the description and label said it "removes the exemption ... for homeless individuals or veterans," but it gates all HR1 ABAWD changes (higher exempted age threshold, lower dependent-child age threshold, exemption removals and additions). Rewritten accurately; references kept.weekly_hours_worked_before_lsris read overperiod.this_year, noting the annual-average proxy for what is legally a monthly determination (20 hrs/week averaged monthly / 80 hrs/month per 7 CFR 273.24(a)(1); 30 hrs/week per 273.7(b)(1)(vii)).is_snap_work_registration_exempt_non_age.py): comment noting that annual unemployment compensation receipt exempting all months of the year is a simplification.The optional parameterized grace-months fraction from the issue is deliberately excluded to keep this PR docs-only; it can follow in a separate PR if wanted.
Testing
policyengine-core test policyengine_us/tests/policy/baseline/gov/usda/snap/eligibility/work_requirements -c policyengine_us— 61 passed, unchanged.🤖 Generated with Claude Code